Skip to content

Conversation

hedrok
Copy link
Contributor

@hedrok hedrok commented Sep 2, 2025

ipt_NETFLOW: fix aggregation mode

Add patch 0004-Change-aggregation-parameter-mode-400-444.patch: Tests need to read 'aggregation' parameter. All other parameters have modes 444 or 644. Change mode of 'aggregation' 400 -> 444

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

* 0004-Change-aggregation-parameter-mode-400-444.patch:
  Tests need to read 'aggregation' parameter. All other parameters have
  modes 444 or 644. Change mode of 'aggregation' 400 -> 444
Copy link

github-actions bot commented Sep 2, 2025

👍
No issues in PR Title / Commit Title

@github-actions github-actions bot added the current VyOS rolling release label Sep 2, 2025
Copy link
Member

@c-po c-po left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we probably upstream this?

@hedrok
Copy link
Contributor Author

hedrok commented Sep 3, 2025

I'll try, but ipt_NETFLOW seems not active unfortunately... My other patch about vlan output is also good for upstream, I'll create PRs for both.

@hedrok
Copy link
Contributor Author

hedrok commented Sep 4, 2025

@sever-sever
Copy link
Member

aabc/ipt-netflow#246 aabc/ipt-netflow#247

@c-po @dmbaturin I think we should fork the repo as there is no activity for months

Copy link
Member

@sever-sever sever-sever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change module mode

@sever-sever sever-sever merged commit c6d3547 into vyos:current Sep 6, 2025
8 checks passed
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Sep 6, 2025
nvollmar pushed a commit to nvollmar/vyos-build that referenced this pull request Sep 8, 2025
…mode

T75: ipt-netflow fix aggregation mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current VyOS rolling release mirror-completed
Development

Successfully merging this pull request may close these issues.

4 participants